Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert "add sepolicy for system_suspend service" #270

Open
wants to merge 1 commit into
base: celadon/s/mr0/master
Choose a base branch
from
Open

Revert "add sepolicy for system_suspend service" #270

wants to merge 1 commit into from
+0 −125

Conversation

qizhangz
Copy link

This reverts commit e7b5d48. Move the change to os.android.bsp.utils-bsp-apollo.

Trackd-On: OAM-113372

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has completed Engineering Build for this issue, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details

1 similar comment
@sysopenci
Copy link

Android CI has completed Engineering Build for this issue, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details

cchen113
cchen113 approved these changes Nov 15, 2023
View reviewed changes
Copy link

@cchen113 cchen113 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

sysopenci
sysopenci reviewed Nov 15, 2023
View reviewed changes
Copy link

@sysopenci sysopenci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failure when executing preintegration builder

FAILURE:CheckApprovals Some checks are failed. See the step results.html

More details, logs, and binaries: /absp/builders/celadon_smr0_master-preintegration/builds/996

@qizhangz
Revert "add sepolicy for system_suspend service"
a2d8d92 
This reverts commit e7b5d48.
Move the change to os.android.bsp.utils-bsp-apollo.

Test Done:
  suspend/resume working;
  aplogs not showing the avc: denied error logs
  for suspend service.

Tracked-On: OAM-113372
Signed-off-by: Qi Zhang <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has completed Engineering Build for this issue, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details

@sysopenci
Copy link

Android CI has completed Engineering Build for this issue, build is SUCCESS.To merge the changes please click on APPROVE FOR MERGE button in Android CI WEB

@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sysopenci sysopenci sysopenci left review comments

@cchen113 cchen113 cchen113 approved these changes

@JeevakaPrabu JeevakaPrabu Awaiting requested review from JeevakaPrabu

@kwang13 kwang13 Awaiting requested review from kwang13

@lchen43 lchen43 Awaiting requested review from lchen43

@sgnanase sgnanase Awaiting requested review from sgnanase

@swaroopbalan swaroopbalan Awaiting requested review from swaroopbalan

@YadongQi YadongQi Awaiting requested review from YadongQi

Assignees
No one assigned
Labels
Stale Stale label for inactive open prs Valid commit message
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@qizhangz @sysopenci @kwang13 @cchen113